Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Delete image not work after rename. #113

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

rb-union
Copy link
Contributor

@rb-union rb-union commented Jan 2, 2024

旧版看图存在多种取得当前图片的接口,
删除图片文件使用的接口所指向的文件路径
变量在重命名后未更新.
调整取得当前图片路径的接口,和其它代码
保持一致.同时更新重命名后当前图片路径
变量用于其它内部处理获取正确信息.

Log: 修复重命名后删除图片无响应.
Bug: https://pms.uniontech.com/bug-view-236589.html
Influence: Delete
Change-Id: I1e54f3f289e3d88ec84afed686ad16bd0137a73f

旧版看图存在多种取得当前图片的接口, 删除图片文件
使用的接口所指向的文件路径变量在重命名后未更新.
调整取得当前图片路径的接口,和其它代码保持一致.
同时更新重命名后当前图片路径变量用于其它内部处理
获取正确信息.

Log: 修复重命名后删除图片无响应.
Bug: https://pms.uniontech.com/bug-view-236589.html
Influence: Delete
Change-Id: I1e54f3f289e3d88ec84afed686ad16bd0137a73f
Copy link
Contributor

@starhcq starhcq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/+1

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rb-union, starhcq

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@starhcq starhcq merged commit 708845f into linuxdeepin:master Jan 3, 2024
17 checks passed
@rb-union rb-union deleted the fix_236589 branch January 3, 2024 05:19
rb-union added a commit to rb-union/image-editor that referenced this pull request Jan 10, 2024
Bump version to 1.0.40
PR:
* linuxdeepin#113

Log: Bump version to 1.0.40
Change-Id: I2810650f972d751b0a14535b8e9b25ca29329a71
rb-union added a commit to rb-union/image-editor that referenced this pull request Jan 10, 2024
Bump version to 1.0.41
PR:
* linuxdeepin#113

Log: Bump version to 1.0.41
starhcq pushed a commit that referenced this pull request Jan 10, 2024
Bump version to 1.0.41
PR:
* #113

Log: Bump version to 1.0.41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants